Skip to content

Fix macOS Electron artifact upload path#871

Merged
Tim020 merged 2 commits intodevfrom
fix/macos-electron-artifact-path
Jan 24, 2026
Merged

Fix macOS Electron artifact upload path#871
Tim020 merged 2 commits intodevfrom
fix/macos-electron-artifact-path

Conversation

@Tim020
Copy link
Copy Markdown
Contributor

@Tim020 Tim020 commented Jan 24, 2026

Summary

  • Fixed glob pattern for macOS Electron build artifacts that was preventing uploads to releases

Problem

The v0.24.0 release was missing the macOS Electron build (.zip). Investigation of the workflow logs showed:

  • macOS build succeeded and produced: electron/out/make/zip/darwin/arm64/DigiScript-darwin-arm64-0.24.0.zip
  • Artifact upload failed with: No files were found with the provided path: electron/out/make/*.zip

Root Cause

Electron Forge outputs macOS builds to a nested directory structure (make/zip/darwin/{arch}/), but the workflow used a non-recursive glob pattern (*.zip) that only matched files directly in make/.

Fix

Changed the artifact pattern from electron/out/make/*.zip to electron/out/make/**/*.zip to recursively find zip files in subdirectories.

Test plan

  • Verify next release includes macOS Electron build artifact
  • Check workflow logs show successful artifact upload for macOS

🤖 Generated with Claude Code

Tim020 and others added 2 commits January 24, 2026 22:32
The glob pattern `electron/out/make/*.zip` did not match the actual
output path from Electron Forge, which places macOS builds at
`electron/out/make/zip/darwin/{arch}/*.zip`.

Changed to recursive glob `**/*.zip` to find zip files in subdirectories.

This fixes the missing macOS Electron build in releases.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Tim020 Tim020 added the claude Issues created by Claude label Jan 24, 2026
@github-actions github-actions bot added github GitHub actions related issue or pull request xsmall-diff labels Jan 24, 2026
@github-actions
Copy link
Copy Markdown

Client Test Results

83 tests  ±0   83 ✅ ±0   0s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 61eade9. ± Comparison against base commit 67ed187.

@github-actions
Copy link
Copy Markdown

Python Test Results

  1 files  ±0    1 suites  ±0   35s ⏱️ ±0s
313 tests ±0  313 ✅ ±0  0 💤 ±0  0 ❌ ±0 
318 runs  ±0  318 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 61eade9. ± Comparison against base commit 67ed187.

@Tim020 Tim020 merged commit 6417260 into dev Jan 24, 2026
24 checks passed
@Tim020 Tim020 deleted the fix/macos-electron-artifact-path branch January 24, 2026 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude Issues created by Claude github GitHub actions related issue or pull request xsmall-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant